Skip to content

Conversation

@Phemisters
Copy link
Contributor

darktable PR: darktable-org/darktable#18547

Update the default in preferences for the workflow - I moved sigmoid up as it is the new default and that also makes the description order correspond to the order of the combo-box.

While doing this I noticed the white balance now has "as shot to reference" and for me that seems to be what it defaults to. That is why I updated the sentence about white balance, as I think it applies in the same way as for "camera reference". You may prefer we wait to do that change when darktable PR 15602 is done.

@elstoc
Copy link
Contributor

elstoc commented Apr 18, 2025

Thanks. I will hold off merging this for now as I suspect we might need to make some other tweaks elsewhere in the doc to account for this. For example, while I'm not currently interested in changing the recommendations in the overview/workflow section (which suggests using filmic to process), it might still need some wording tweaks to account for the new default.

@Phemisters
Copy link
Contributor Author

we might need to make some other tweaks

I'm not sure if you have a plan for this. I have had a go at some small updates for this, but happy to revert those or indeed close this PR until someone has the time and knowledge for a more thorough update.

@github-actions
Copy link

This pull request has not had any activity in the past 60 days and will be closed in 365 days if not updated. Please verify it has no conflicts with the master branch and rebase if needed. Please add a comment if you need help or give permission to other people to finish your work.

@github-actions github-actions bot added the no-pr-activity No activity on this PR label Jun 26, 2025
@Phemisters Phemisters force-pushed the dt-default-tone-mapper-doc branch from 1ae7cfc to 937284c Compare September 17, 2025 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-pr-activity No activity on this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants